Skip to content

docs: remove multi-tenancy feature-flag (#280)#330

Merged
RomuDeuxfois merged 1 commit into
mainfrom
multi-tenancy/ff-removal
Jun 19, 2026
Merged

docs: remove multi-tenancy feature-flag (#280)#330
RomuDeuxfois merged 1 commit into
mainfrom
multi-tenancy/ff-removal

Conversation

@RomuDeuxfois

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings June 19, 2026 12:12
@RomuDeuxfois RomuDeuxfois changed the title [docs] remove multi-tenancy feature-flag [docs] remove multi-tenancy feature-flag (#280) Jun 19, 2026
@Filigran-Automation Filigran-Automation added the filigran team Item from the Filigran team. label Jun 19, 2026
@Filigran-Automation Filigran-Automation changed the title [docs] remove multi-tenancy feature-flag (#280) docs: remove multi-tenancy feature-flag (#280) Jun 19, 2026
@Filigran-Automation

Copy link
Copy Markdown

🤖 [AI-generated]

Hey @RomuDeuxfois! 👋 Thanks a lot for opening PR #330 — really appreciate you keeping the docs in sync! 🙏

I just had a quick look and noticed the description is currently empty. I haven't changed anything — just a gentle, optional suggestion to add a couple of lines so reviewers can get through it faster:

Area What could help Suggestion
Description The body is empty A sentence or two on what changed and why — e.g. that the multi-tenancy feature-flag is being removed from the docs (and the related mkdocs.yml entry), and why (the flag is gone / GA'd)
Linked issue Context for reviewers The title already references #280 — a quick Closes #280 in the body makes the link explicit too

💡 If helpful, the contribution conventions walk through what to include.

No rush at all — thanks again for contributing to OpenAEV! 🚀

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the documentation feature-flag gating for the Multi-tenancy page so the content is always rendered, and updates the example feature-flag list accordingly in the MkDocs configuration.

Changes:

  • Remove the {% if "multi-tenancy" ... %} / {% endif %} wrapper from docs/administration/multi-tenancy.md.
  • Remove the “Under construction” admonition from the Multi-tenancy page header.
  • Update the commented example of DOCS_FEATURE_FLAGS in mkdocs.yml to no longer include multi-tenancy.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
mkdocs.yml Updates the example DOCS_FEATURE_FLAGS comment to remove multi-tenancy.
docs/administration/multi-tenancy.md Makes the Multi-tenancy documentation page unconditional (no feature-flag gating) and removes the “Under construction” notice.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@RomuDeuxfois RomuDeuxfois merged commit e258bb4 into main Jun 19, 2026
3 of 5 checks passed
@RomuDeuxfois RomuDeuxfois deleted the multi-tenancy/ff-removal branch June 19, 2026 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran team Item from the Filigran team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants